[Obs AI Assistant] Add route privilege tests for Serverless #205210
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
viduni94
added
release_note:skip
|
Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant) |
viduni94
force-pushed
the
add-route-privilege-tests-to-serverless
branch
from
ed525be to
c085431
Compare
dgieselaar
reviewed
Dec 30, 2024
| @@ -170,57 +172,26 @@ export default function ApiTest({ getService }: FtrProviderContext) { | |||
| ]); | |||
| }); | |||
|
|
|||
| it.skip('returns a useful error if the request fails', async () => { | |||
There was a problem hiding this comment.
It was skipped as part of the migration to inference endpoints. The discussion thread can be found here - https://github.com/elastic/kibana/pull/199286/files#r1867485436
dgieselaar
reviewed
Dec 30, 2024
...rless/api_integration/test_suites/observability/ai_assistant/tests/complete/complete.spec.ts
Outdated
Show resolved
Hide resolved
viduni94
force-pushed
the
add-route-privilege-tests-to-serverless
branch
from
c085431 to
8852cdc
Compare
viduni94
force-pushed
the
add-route-privilege-tests-to-serverless
branch
from
21f980b to
df3715d
Compare
neptunian
approved these changes
Jan 2, 2025
7 tasks
sorenlouv
reviewed
Jan 3, 2025
...s/api_integration/test_suites/observability/ai_assistant/tests/connectors/connectors.spec.ts
Outdated
Show resolved
Hide resolved
sorenlouv
approved these changes
Jan 3, 2025
viduni94
force-pushed
the
add-route-privilege-tests-to-serverless
branch
from
493d171 to
6beb135
Compare
💚 Build Succeeded
Metrics [docs]
History
cc @viduni94 |
|
Starting backport for target branches: 8.x |
kibanamachine
pushed a commit
to kibanamachine/kibana
that referenced
this pull request
Jan 3, 2025
…205210) Closes elastic#204884 ## Summary This PR adds security and route privilege tests to the serverless test suite. ### Checklist - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios - [x] [Flaky Test Runner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was used on any tests changed - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) (cherry picked from commit 7d76276)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
kibanamachine
added a commit
that referenced
this pull request
Jan 3, 2025
…05210) (#205517) # Backport This will backport the following commits from `main` to `8.x`: - [[Obs AI Assistant] Add route privilege tests for Serverless (#205210)](#205210) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Viduni Wickramarachchi","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-03T16:17:43Z","message":"[Obs AI Assistant] Add route privilege tests for Serverless (#205210)\n\nCloses https://github.com/elastic/kibana/issues/204884\n\n## Summary\n\nThis PR adds security and route privilege tests to the serverless test\nsuite.\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7d76276d8b1988599916f59052f7ebd9e815a3c5","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Obs AI Assistant","backport:version","v8.18.0"],"title":"[Obs AI Assistant] Add route privilege tests for Serverless","number":205210,"url":"https://github.com/elastic/kibana/pull/205210","mergeCommit":{"message":"[Obs AI Assistant] Add route privilege tests for Serverless (#205210)\n\nCloses https://github.com/elastic/kibana/issues/204884\n\n## Summary\n\nThis PR adds security and route privilege tests to the serverless test\nsuite.\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7d76276d8b1988599916f59052f7ebd9e815a3c5"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205210","number":205210,"mergeCommit":{"message":"[Obs AI Assistant] Add route privilege tests for Serverless (#205210)\n\nCloses https://github.com/elastic/kibana/issues/204884\n\n## Summary\n\nThis PR adds security and route privilege tests to the serverless test\nsuite.\n\n### Checklist\n\n- [x] [Unit or functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere updated or added to match the most common scenarios\n- [x] [Flaky Test\nRunner](https://ci-stats.kibana.dev/trigger_flaky_test_runner/1) was\nused on any tests changed\n- [x] The PR description includes the appropriate Release Notes section,\nand the correct `release_note:*` label is applied per the\n[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)","sha":"7d76276d8b1988599916f59052f7ebd9e815a3c5"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Viduni Wickramarachchi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #204884
Summary
This PR adds security and route privilege tests to the serverless test suite.
Checklist
release_note:*label is applied per the guidelines